Revert "level-bar: introduce GtkLevelBar"

This reverts commit 126a2308ca467744178d4be3309403f6899de987.

Pushed by mistake.

Revert "visuals: add a visual demo for GtkLevelBar"

This reverts commit ffdd7a5e185879006d405ffd69f68ff60f5fccfa.

Pushed by mistake.

docs: link against libgdk-3.la

https://bugzilla.gnome.org/show_bug.cgi?id=679409

visuals: add a visual demo for GtkLevelBar

https://bugzilla.gnome.org/show_bug.cgi?id=677892

https://bugzilla.gnome.org/show_bug.cgi?id=679409

level-bar: introduce GtkLevelBar

Similar to CcStrengthBar from gnome-control-center, but more generic and
with thorough CSS styling support.

https://bugzilla.gnome.org/show_bug.cgi?id=677892

GtkApplication: Fix unsetting menu models

Pointed out by Zachary Dovel in
http://bugzilla.gnome.org/show_bug.cgi?id=679705

Updated Norwegian nynorsk translation

Silence the build

Hide gdbus-codegen behind $(AM_V_GEN) for silence and cleanliness.

Add missing functions to the docs

://bugzilla.gnome.org/show_bug.cgi?id=678828

OpenBSD: use G_N_ELEMENTS instead of nitems

nitems is never guaranteed to be defined in sys/params.h as it is meant
to be defined within a protected ifdef __KERNEL conditional.
Use G_N_ELEMENTS from GLib which is the proper way of doing this.

a11y: Make lockbutton notify about state changes

In particular, ensure that the name of the lockbutton gets updated as
the displayed text of the lockbutton changes.

https://bugzilla.gnome.org/show_bug.cgi?id=677347

a11y: Get lockbutton text directly

Instead of letting the generic button code attempt to find the currently
displayed text, just return the text directly.

https://bugzilla.gnome.org/show_bug.cgi?id=677347

a11y: Add separate accessible for GtkLockButton

The accessible doesn't do anything yet, this is just infrastructure
work.

a11y: Don't mess up if column gets hidden

We were using a function that insisted on visible columns for computing
the potential ID of an invisible column. Not good.

https://bugzilla.gnome.org/show_bug.cgi?id=675990

entry: guard for != NULL before unreffing the pixbuf

In gtk_entry_get_icon_pixbuf() we unreference the pixbuf that
_gtk_icon_helper_ensure_pixbuf() gives us back, since the function
doesn't return a reference, and by doing so we're able to return the
reference owned by the cached icon helper.
Since the icon helper method can return NULL though, if no icon
properties have been set on it, guard for != NULL before unreffing the
pixbuf, as that would cause a critical warning.

https://bugzilla.gnome.org/show_bug.cgi?id=679537

modelmenu: listen for toplevel changes on the attach widget

Right now, when we create a GtkModelMenu for a GMenuModel, we listen to
changes to the menu's attach-widget to detect when a toplevel
GtkApplicationWindow becomes available to fetch actions from it.

This unfortunately breaks this simple code:

  GtkWidget *application_window = gtk_application_window_new();
  GtkWidget *menu_button = gtk_menu_button_new();
  GMenuModel *menu_model = get_menu_model();

  gtk_menu_button_set_menu_model(menu_button, menu_model);
  gtk_container_add(GTK_CONTAINER(application_window), menu_button);

Since GtkMenuButton creates a GtkModelMenu and sets itself as its attach
widget before it's added to a hierarchy containing a
GtkApplicationWindow.

Fix the bug by simply listening for changes in the window hierarchy, and
creating the menu model when the attach widget is added to an
application window.

https://bugzilla.gnome.org/show_bug.cgi?id=679454

menu: notify attach-widget property when menu is detached

When the menu is detached, the attach-widget property changes value to
NULL, so we should notify a property change, like
gtk_menu_attach_to_widget() does.

https://bugzilla.gnome.org/show_bug.cgi?id=679454

application: fix double free of app_id

This is fallout from commit 257b42e2f9d0759cc79967034f80c571c600faf3 -
those fields were already getting freed in
gtk_application_shutdown_x11() and my commit caused crashes on quit
instead.

Thanks to Rico Tzschichholz for reporting the bug and testing this fix.

modelmenu: set the accel group on the GtkMenu

Or the accelerators added for actions in the menu won't be displayed in
the menu items.

https://bugzilla.gnome.org/show_bug.cgi?id=679166

menubutton: set DROPDOWN_MENU hint on the menu toplevel

So the WM can render the menu shadows correctly, omitting the top side,
in the common case the menu is rendered down from a toolbar.

https://bugzilla.gnome.org/show_bug.cgi?id=679449

gtkstyle: add a gtkstyle-fallback style class for RC colors

As an addition to 10423726709539724be0ea19bed76ba4331af774, themes might
want to avoid using the hardcoded GTK defaults for legacy GtkStyle
values. Add a gtkstyle-fallback style class that can be used by themes
to tweak the legacy GtkStyle defaults.

https://bugzilla.gnome.org/show_bug.cgi?id=679092

application: plug some memory leaks

docs: fix a number of typos and obsolete references

Updated Vietnamese translation

po/vi: import from Damned Lies

menubutton: don't dereference NULL GdkEvent

Since we explictly call popup_menu with NULL when it's
keyboard-activated, we need to be careful and not access event->device
without checking for event != NULL before.

docs: add missing GtkApplication accelerator methods to doc sections

GActionMuxer: disconnect group signals on finalize

The signals for the action group were being disconnected when the action
group was explicitly removed from the GActionMuxer but the same was not
being done when it was finalized.

This means that a change in the state of an action group that used to be
associated with a finalized GActionMuxer would result in a crash.  This
would happen for stateful application actions after closing a window.

docs: add gtk_ui_manager_add_ui_from_resource to gtk-doc

Or it won't show up in devhelp/online dev docs.

Handle transparent colors better in GtkStyle emulation

We can't expose colors with alpha in the GtkStyle colors, since
GdkColor has no alpha. Currently we throw away alpha completely,
which fails very badly for completely transparent backgrounds,
which now is the default for most widgets, as it typically
end up with black-on-black.

We handle this by falling back on the default/previous colors for
transparent colors. This is is simple and avoids complete failure.

https://bugzilla.gnome.org/show_bug.cgi?id=671437

Updated Greek translation

a11y: remove implementation for [add/remove]_global_event_listener

Since release 2.5.3, ATK provides an implementation for those
methods, so toolkits doesn't need to implement them.

Updated Telugu Translation

Updated Bulgarian translation

build: Drop --without-atk-bridge option

Instead, always build it if and only if X11.  This reduces the set of
supported configurations.

https://bugzilla.gnome.org/show_bug.cgi?id=677491

widget-factory: add a symbolic icon column in the treeview example

Bump version

3.5.6

Don't export private GtkMountOperationHandler api

Updates for 3.5.6

radiobutton: add missing (element-type) annotation for radio groups

The GSLists should specify their element-type in every annotation. We
are missing some.

https://bugzilla.gnome.org/show_bug.cgi?id=677127

menu: add missing (scope async) annotation to gtk_menu_popup()

Or the g-i scanner will emit a warning.

https://bugzilla.gnome.org/show_bug.cgi?id=677127

widget: silence g-i scanner warning

The parameter should be named the same in the definition and the
declararion.

https://bugzilla.gnome.org/show_bug.cgi?id=677127

fontchooser: silence g-i scanner warning

These parameters should be named the same in the definition and the
declaration.

https://bugzilla.gnome.org/show_bug.cgi?id=677127

actionable: silence g-i scanner warning

The parameter should be named the same in the definition and the
declararion.

https://bugzilla.gnome.org/show_bug.cgi?id=677127

application: fix a typo in introspection annotation

https://bugzilla.gnome.org/show_bug.cgi?id=677127

Bug 676077: Fix handling of Keyboard Input on Windows

Don't use ASCII control characters to denote the input of Esc, Tab,
Return/Enter, Backspace and Delete, as it seems that it is not how
Windows handle them, and they cause weird characters to appear in the
input field on GTK+3 programs in non-English Windows.  Instead, let
these keys be handled as-is on Windows, like what is done in GTK+-2.x.

Checked with mclasen on IRC, and thanks to the people who verified the
patch to not break anything on English Windows.

Rename gtk_icon_view_get_cell_area to gtk_icon_view_get_cell_rect

get_cell_area was "conflicting" with the property named "cell-area"
which is something different.

Updated Galician translations

Enable gio-unix for Quartz

Fixes a compile failure because gdbus-config uses G_OS_UNIX to guard
gio-unix features instead of HAVE_GIO_UNIX.

Cosmetics

Fix a compiler warning

This was introduced in the previous commit.

GtkNotebook: Fix reverse tab order

When using Shift-Tab to move the focus out of page content onto
the tab label, we end up in a situation where both Tab and Shift-Tab
move focus back into the page, which is not really what is expected
when the notebook is part of a dialog.

Instead, arrange things so that using Shift-Tab with focus on a
tab label moves the focus out of the notebook.

http://bugzilla.gnome.org/show_bug.cgi?id=669986

gtkdbusinterfaces: Fix srcdir != builddir

gtk: add gtkdbusinterfaces.xml to EXTRA_DIST

GtkNotebook: Maintain invariants during tab dnd

It turns out that we can end up removing a notebook child while
the tab is still 'detached'. Child removal causes
gtk_notebook_remove_tab_label() to be called on the tab label,
but that function did not deal with the eventuality that the tab
label may be a child of the dnd window.

http://bugzilla.gnome.org/show_bug.cgi?id=677943

config.h.win32.in: Updates

Make it reflect the contents of the config.h.in template.

gtk/gtkcsseasevalue.c: Remove C99ism

Update Visual C++ property sheets

"Install" the demo css files as well.

Fix the gtk3-demo Visual C++ projects

Add the demos that were introduced recently in the projects.

gtk/Makefile.am: Fix Visual C++ disthook rules

gtkdbusgenerated.c needs to be included in the projects as well.

Update Visual C++ 2008 property sheet

Standardize on using $(CopyDir), as it is now done in the Visual C++ 2010
sheets, for consistency reasons.

build: Add --without-atk-bridge, only check for it on X11 platforms

Some builders using gtk3 outside of the GNOME cycle want an option to
avoid linking to atk-bridge-2.0.  Provide that, and at the same time
ensure we're only looking for it on X11 platforms.

https://bugzilla.gnome.org/show_bug.cgi?id=677491

mountoperation: use the Shell DBus proxy if available

Make GMountOperation look for an owner of org.Gtk.MountOperationHandler
if possible, and use it instead of the GTK-based dialogs.
This allows applications to use the implementation offered by the
desktop shell, if available, through a DBus private interface:
org.Gtk.MountOperationHandler.

https://bugzilla.gnome.org/show_bug.cgi?id=674963

Add gtk_icon_view_get_cell_area

This gets the current cell area of a particular item. Its similar
to gtk_tree_view_get_cell_area().

The code is extracted from gtk_icon_view_set_tooltip_cell which now
just calls the old code.

https://bugzilla.gnome.org/show_bug.cgi?id=678418

Fixed the issue reported in Bug #613561

GdkDevice: Fix an inverted assertion

gdk_device_list_slave_devices only makes sense to call on master
devices, yet its g_return_if_fail check made it reject such devices.

Pointed out by monty.

a11y tests: update expected results

Updated Telugu Translation

Fix a memory leak in GtkImage

https://bugzilla.gnome.org/show_bug.cgi?id=678175

Updated Spanish translation

GtkImage: fix get_property('stock')

The code was trying to get icon's name instead of icon's stock-id.

https://bugzilla.gnome.org/show_bug.cgi?id=678407

Fix windres invocation

Signed-off-by: Chun-wei Fan <fanchunwei@src.gnome.org>

widget: don't call gtk_style_context_set_background if app_paintable

When a widget is app_paintable, its background should not be drawn by
the theme, thus we should not try to override its background again when
style-updated is fired.
This is a bit of a hack, but it fixes gray surfaces observed for DnD
windows with recent GTK+.

Updated Arabic translation

Updated POTFILES.in

gtkmenutoolbutton: Use GtkMenuButton

To implement the drop-down menu.

https://bugzilla.gnome.org/show_bug.cgi?id=668013

gtkmenubutton: Add menu button widget

As used in Totem and gnome-contacts. The widget
takes either a GtkMenu or a GMenuModel to construct
its menu, and can be given a parent widget to use to
position the drop-down (as used in GtkMenuToolButton).

https://bugzilla.gnome.org/show_bug.cgi?id=668013

gtk: Always load the atk-bridge

Slightly modified by Alejandro Piñeiro

https://bugzilla.gnome.org/show_bug.cgi?id=677491

image: Don't add border twice

Failed to delete these lines in d40c496e6eacb9f95c2b27eaebf58d320c2b9387

reftests: Add a reftest for the empty iconview

iconview: Special case the empty iconview

If the icon view is empty, we cannot get a reasonable size request from
the cell renderers. So all values we would compute are pretty much
useless.
So we special case it.

This also gets rid of a bunch of crashers from div-by-0 in corner cases.

https://bugzilla.gnome.org/show_bug.cgi?id=677809

image: background and border should be drawn around the widget

... not around the contents. If you want the widget to not be larger
than the contents, don't use ALIGN_FILL.

iconview: Add a descriptive function

I want to special-case empty icon views in the sizing code to avoid
various bugs, and having such a function is nice.

iconview: Ensure icons are at least 1x1 pixels

This is useful for 2 reasons:
(1) Items actually exist and are clickable
(2) Size computations don't divide by 0

I've not seen problems with this in the wild (mostly because
item-padding defaults to non-0), but noticed this while fixing other
bugs.

iconview: Don't add item-padding

... to item sizes when there are no items.

iconview: clamp item size to be in [minimum, natural]

Minimum size is necessary so you can see the item. If we can't get that
we need to scroll.
Natural as the maximum is used so that the spacing between items doesn't
increase when resizing the iconview, but empty space is added to the
right/bottom instead.

https://bugzilla.gnome.org/show_bug.cgi?id=677809

iconview: Include column spacing in calculation

The function is probably almost never used, so nobody has ever seen this
bug, but we should still get it right.

https://bugzilla.gnome.org/show_bug.cgi?id=677809

iconview: Add documentation abour different sizes

Did I mention I hate code that has different kinds of "boxes" or
coordinate systems but no clear way to distinguish them?
It's all ints here...

https://bugzilla.gnome.org/show_bug.cgi?id=677809

iconview: gtk_icon_view_compute_n_items_for_size bugfix

While working on the "iconview: Don't shrink items" patch I noticed that
gtk_icon_view_compute_n_items_for_size modifies the natural and minimum
item sizes it got from gtk_icon_view_get_preferred_item_size when
calculating the max number of items which will fit, but later on it
checks against these sizes when calculating the item_size, and these
checks expect these values to be unmodified.

This patch fixes this by modifying the natural and minimum values in
advance and doing all computations with modified values.

https://bugzilla.gnome.org/show_bug.cgi?id=677809

iconview: Don't shrink items

The previous code assumed that the width was always enough for more than
one column, which is obviously not correct when a number of columns is
hardcoded.

With this patch, it will now always check that the width is enough and
otherwise cause scrolling.

https://bugzilla.gnome.org/show_bug.cgi?id=677809

toolbar: don't call gtk_style_context_set_background()

GtkToolbar doesn't have its own GdkWindow to draw on (it calls
gtk_widget_set_has_window(FALSE) in _init), but only an event window
(input-only).
Since gtk_widget_get_window() in that case will return the GdkWindow of
the parent container, by calling gtk_style_context_set_background() here
we're overriding the base background of the container instead of our.
While in most cases this doesn't have any noticeable effect, since
the toplevel GtkWindow will paint its background on top of it at the
beginning of the draw cycle, when the classic window hierarchy is
broken, such as when widgets are rendered through a clutter-gtk
offscreen embedding, the background will become visible, which is
undesirable.
Fix this by having GtkToolbar not call gtk_style_context_set_background
in its style_updated handler.

Fix up the deprecation annotation for gtk_widget_get_pointer

When I added the versioned annotation, I accidentally backdated
it, so the Deprecated: tag in the docs said 3.4, but the annotation
said 3.0. Fix it so we say 3.4 in both places.

assistant: Set the proper stock images on the proper buttons

Copy/paste typo.

entry-completion: don't move to leftmost character with inline-selection

When inline-selection is set, and the completion popup is showing,
pressing left abruptly jumps to the beginning of the entry text.
This is not expected, since the cursor is at the end of the text before
the left key is pressed, and this behavior is completely inconsistent
with how an entry would normally behave.
The behavior can be observed in Epiphany by selecting a completion match
and pressing left.

This patch changes the code so that it just runs the default entry key
press keybindings in such a case.

https://bugzilla.gnome.org/show_bug.cgi?id=677915

stylecontext: pass down the requested change type to invalidation

<Company>: cosimoc: yes, that totally makes sense
<Company>: looks like a copy/paste error from when i factored out that
           function

fix  a typo

liststore: Make gtk_list_store_reorder() introspectable

Mark the new_order argument as zero-terminated array, even though it does not
need to be zero terminated (it has an implicit length not given by a constant
or another method argument). It does not hurt if bindings append an extra zero
to the array as long as it has enough elements, and this makes the method
introspectable.

https://bugzilla.gnome.org/show_bug.cgi?id=677941

application: Really allow NULL application-id

While the docs were adjusted to claim that NULL application-id is allowed,
the code actually segfaulted on it.

testscale: add a test for up/leftwards scales